Simplification bienvenue — Remplacer le LOG_CONFIG dict + log_config=None par un simple logging.basicConfig() est la bonne approche. Moins de code, moins d'indirection, plus lisible.

Cohérence entre projets — Le pattern correspond exactement à ce qui est fait dans mcp-raganything/main.py (même format, même datefmt). C'est le bon réflexe pour homogénéiser le logging across les services SoluDevTech.

Format de log amélioré — Le passage de %(asctime)s - %(name)s - %(levelname)s - %(message)s à %(asctime)s | %(levelname)-8s | %(name)s | %(message)s est un upgrade :

• Les | sont plus lisibles que les - (séparateur visuel clair)
• %(levelname)-8s avec alignement fixe → colonnes visuellement alignées dans les logs
• Le datefmt explicite %Y-%m-%d %H:%M:%S élimine le format par défaut moche de Python

Suppression de force=True — C'est correct. force=True est utile quand tu dois reconfigurer un logger déjà existant, mais ici au startup c'est le premier appel, pas besoin.

Suppression de import sys et stream=sys.stdout — Propre. basicConfig écrit sur stderr par défaut, mais uvicorn redirige déjà correctement les logs. Pas besoin de forcer stdout.

Niveau de log hardcodé vs. settings — level=logging.INFO est hardcodé alors que settings.log_level existe et est utilisé pour uvicorn. Dans mcp-raganything, le pattern est level=app_config.UVICORN_LOG_LEVEL.upper(). Ici, si quelqu'un met LOG_LEVEL=DEBUG dans l'env, les logs Python resteront à INFO mais uvicorn sera en DEBUG. Incohérent.

Suggestion :

logging.basicConfig(
    level=getattr(logging, settings.log_level.upper(), logging.INFO),
    format="%(asctime)s | %(levelname)-8s | %(name)s | %(message)s",
    datefmt="%Y-%m-%d %H:%M:%S",
)

Suppression de logging.getLogger().setLevel(logging.INFO) — OK si basicConfig suffit, mais attention : cette ligne était un filet de sécurité pour forcer le root logger à INFO même si une lib tierce l'avait changé. Si une dépendance importée avant ce fichier configure le root logger, basicConfig ne fait rien (il ne s'applique que si aucun handler n'existe encore). Le force=True d'origine gérait ce cas.

Suggestion alternative : Garder force=True si tu veux garantir la config même en cas d'imports parasites :

logging.basicConfig(
    level=getattr(logging, settings.log_level.upper(), logging.INFO),
    format="%(asctime)s | %(levelname)-8s | %(name)s | %(message)s",
    datefmt="%Y-%m-%d %H:%M:%S",
    force=True,
)

Pas de newline à la fin du fichier — Le diff montre \ No newline at end of file. Ajoute un saut de ligne final, c'est une convention POSIX et la plupart des éditeurs/linters le signalent.

Simplification bienvenue — Remplacer le LOG_CONFIG dict + log_config=None par un simple logging.basicConfig() est la bonne approche. Moins de code, moins de 'indirection, plus lisible.

Cohérence entre projets — Le pattern correspond exactement à ce qui est fait dans mcp-raganything/main.py (même format, même datefmt). C'est le bon réflexe pour homogénéiser le logging across les services SoluDevTech.

Format de log amélioré — Le passage de %(asctime)s - %(name)s - %(levelname)s - %(message)s à %(asctime)s | %(levelname)-8s | %(name)s | %(message)s est un upgrade :

• Les | sont plus lisibles que les - (séparateur visuel clair)
• %(levelname)-8s avec alignement fixe → colonnes visuellement alignées dans les logs
• Le datefmt explicite %Y-%m-%d %H:%M:%S élimine le format par défaut moche de Python

Suppression de force=True — C'est correct. force=True est utile quand tu dois reconfigurer un logger déjà existant, mais ici au startup c'est le premier appel, pas besoin.

Suppression de import sys et stream=sys.stdout — Propre. basicConfig écrit sur stderr par défaut, mais uvicorn redirige déjà correctement les logs. Pas besoin de forcer stdout.

Niveau de log hardcodé vs. settings — level=logging.INFO est hardcodé alors que settings.log_level existe et est utilisé pour uvicorn. Dans mcp-raganything, le pattern est level=app_config.UVICORN_LOG_LEVEL.upper(). Ici, si quelqu'un met LOG_LEVEL=DEBUG dans l'env, les logs Python resteront à INFO mais uvicorn sera en DEBUG. Incohérent.

Suggestion :

logging.basicConfig(
    level=getattr(logging, settings.log_level.upper(), logging.INFO),
    format="%(asctime)s | %(levelname)-8s | %(name)s | %(message)s",
    datefmt="%Y-%m-%d %H:%M:%S",
)

Suppression de logging.getLogger().setLevel(logging.INFO) — OK si basicConfig suffit, mais attention : cette ligne était un filet de sécurité pour forcer le root logger à INFO même si une lib tierce l'avait changé. Si une dépendance importée avant ce fichier configure le root logger, basicConfig ne fait rien (il ne s'applique que si aucun handler n'existe encore). Le force=True d'origine gérait ce cas.

Suggestion alternative : Garder force=True si tu veux garantir la config même en cas d'imports parasites :

logging.basicConfig(
    level=getattr(logging, settings.log_level.upper(), logging.INFO),
    format="%(asctime)s | %(levelname)-8s | %(name)s | %(message)s",
    datefmt="%Y-%m-%d %H:%M:%S",
    force=True,
)

Pas de newline à la fin du fichier — Le diff montre \ No newline at end of file. Ajoute un saut de ligne final, c'est une convention POSIX et la plupart des éditeurs/linters le signalent.